docs: state local MCP key requirements exactly - #251
Open
MagMueller wants to merge 3 commits into
Open
MagMueller wants to merge 3 commits into
MagMueller wants to merge 3 commits into
Conversation
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
Contributor
There was a problem hiding this comment.
1 issue found across 2 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="docs/open-source/llms-full.txt">
<violation number="1" location="docs/open-source/llms-full.txt:2990">
P3: The new OPENAI_API_KEY line says the key is 'Required for extraction', but the Available Tools list groups browser_get_html and browser_screenshot under 'Content Extraction' even though they need no key (as the Model keys table states). A reader who uses get_html/screenshot and sees them under 'Content Extraction' may conclude they need the key. Narrow the wording to the one tool that requires it.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
|
|
||
| - `OPENAI_API_KEY` - Your OpenAI API key (required) | ||
| - `ANTHROPIC_API_KEY` - Your Anthropic API key (alternative to OpenAI) | ||
| - `OPENAI_API_KEY` - Required for extraction and the default autonomous agent. Omit it from the client configurations above if you only use direct browser tools |
Contributor
There was a problem hiding this comment.
P3: The new OPENAI_API_KEY line says the key is 'Required for extraction', but the Available Tools list groups browser_get_html and browser_screenshot under 'Content Extraction' even though they need no key (as the Model keys table states). A reader who uses get_html/screenshot and sees them under 'Content Extraction' may conclude they need the key. Narrow the wording to the one tool that requires it.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At docs/open-source/llms-full.txt, line 2990:
<comment>The new OPENAI_API_KEY line says the key is 'Required for extraction', but the Available Tools list groups browser_get_html and browser_screenshot under 'Content Extraction' even though they need no key (as the Model keys table states). A reader who uses get_html/screenshot and sees them under 'Content Extraction' may conclude they need the key. Narrow the wording to the one tool that requires it.</comment>
<file context>
@@ -2973,12 +2973,42 @@ Add to `~/.codeium/windsurf/mcp_config.json`:
## Environment Variables
-- `OPENAI_API_KEY` - Optional for direct browser control; required for `browser_extract_content` and the default OpenAI-compatible route of `retry_with_browser_use_agent`
+- `OPENAI_API_KEY` - Required for extraction and the default autonomous agent. Omit it from the client configurations above if you only use direct browser tools
- `BROWSER_USE_HEADLESS` - Set to `false` to show browser window
- `BROWSER_USE_DISABLE_SECURITY` - Set to `true` to disable browser security features
</file context>
Suggested change
| - `OPENAI_API_KEY` - Required for extraction and the default autonomous agent. Omit it from the client configurations above if you only use direct browser tools | |
| - `OPENAI_API_KEY` - Required for `browser_extract_content` and the default autonomous agent. Omit it from the client configurations above if you only use direct browser tools |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clarifies that direct local MCP browser tools do not need an LLM key. Content extraction uses an OpenAI-compatible key. The autonomous agent tool uses that route by default and also supports configured AWS Bedrock credentials. Removes the unsupported Anthropic-key fallback claim.
Validated the absent-OpenAI/Anthropic-only error and the Bedrock routing branch without provider calls. The Bedrock constructor was replaced by a sentinel for routing only; AWS authentication and inference were not tested. Mint found no broken links, docs.json parsed, and git diff --check passed. The generated LLM-readable mirror was synchronized.
Summary by cubic
Clarifies the local MCP server's LLM key requirements so users don't set up unused keys. Direct browser automation tools like navigate, click, type, page state, HTML, and screenshot work without an LLM key; content extraction requires an OpenAI-compatible key, and the default autonomous agent route also uses OpenAI-compatible by default or AWS Bedrock when configured. Removes the unsupported Anthropic-key fallback from environment variables and troubleshooting.
Written for commit 0dd6694. Summary will update on new commits.